Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests(eip7742): initial tests for eip7742 #986

Merged
merged 6 commits into from
Dec 4, 2024

Conversation

spencer-tb
Copy link
Contributor

@spencer-tb spencer-tb commented Dec 3, 2024

🗒️ Description

Adds some basic tests for EIP-7742:

  • Test duplicates for the invalid Cancun blob tests that are now valid for Prague.
  • Test for zero blobs in a type 3 txs.
  • Tests for some arbitrary values above the Cancun MAX_BLOBS_PER_BLOCK.
  • Tests with multiple blocks using varied blobs per tx.
  • Tests with multiple blocks ascending from 0 to N.
  • Tests with multiple blocks descending from N to 0.
  • Tests with multiple blocks ascending from 0 to N then descending from N to 0.

We should still add fork transition tests in the future, where we test invalid blob counts before the fork (i.e 256/0 at Cancun). And consider adding some more tests ported from Cancun.

Fills with the following EELS resolution change for Prague:

    "Prague": {
        "git_url": "https://github.com/spencer-tb/execution-specs.git",
        "branch": "forks/prague",
        "commit": "857c3fa4d733535f32e9f17250d5790dd7ab5a76"
    }

🔗 Related Issues

Requires #931.

✅ Checklist

  • All: Set appropriate labels for the changes.
  • All: Considered squashing commits to improve commit history.
  • All: Added an entry to CHANGELOG.md.
  • All: Considered updating the online docs in the ./docs/ directory.

@spencer-tb spencer-tb merged commit e2ca766 into pectra-devnet-5 Dec 4, 2024
1 of 6 checks passed
@spencer-tb spencer-tb deleted the pectra-7742-tests branch December 4, 2024 18:27
@spencer-tb spencer-tb restored the pectra-7742-tests branch December 4, 2024 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fork:prague Prague hardfork scope:tests Scope: Test cases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant